Skip to content

Conversation

@yuuki
Copy link
Owner

@yuuki yuuki commented Jun 25, 2025

Summary

  • Enhanced help documentation to clarify what the CNT field represents in tcpulse output
  • Added explicit formulas showing how CNT is calculated for different connection modes
  • Included practical example for sending exactly one message

Background

Users were confused when using --connections 1 and seeing CNT=500 in the output, expecting CNT to represent the number of connections rather than total messages sent.

Changes

  • Updated --rate flag description: Now includes CNT calculation formulas for both persistent and ephemeral modes
  • Added "Output Format" section in help text explaining:
    • CNT = total number of messages/requests sent (not connections)
    • Persistent mode: CNT = rate × duration × connections
    • Ephemeral mode: CNT = rate × duration
  • Added practical example: Shows how to send exactly 1 message using --connections 1 --rate 1 --duration 1s

Test plan

  • Built successfully with make build
  • Verified help text displays correctly with ./tcpulse -h
  • Confirmed formulas match actual behavior in code (client.go:156 and client.go:208)

🤖 Generated with Claude Code

yuuki and others added 2 commits June 26, 2025 00:04
- Update --rate flag description to explain CNT calculation formula
- Add "Output Format" section explaining CNT represents total messages sent
- Include formulas for persistent (rate × duration × connections) and ephemeral (rate × duration) modes
- Add example showing how to send exactly 1 message

Fixes confusion where users expect CNT to represent number of connections
rather than total messages/requests sent.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Add explanation that CNT represents total messages sent, not connections
- Include CNT calculation formulas for persistent and ephemeral modes
- Update help output example to match current implementation
- Clarify JSON Lines count field corresponds to CNT column

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@yuuki yuuki merged commit 8699597 into main Jun 25, 2025
1 check passed
@github-actions github-actions bot mentioned this pull request Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants